@adeebshihadeh sorry for the ping but this would make our life nicer on hev platforms <3

Where did the old value come from? How many platforms have been checked? Have all three of EV, PHEV, and HEV been checked?

I have same car as DevTekVe and would like this change also.

This was my video from above sunny has inputted here to commit to master

No real source for the old values, it was copied from CF_Lvr_Gear in the OG port: commaai/openpilot#1362, commit: commaai/openpilot@e43fa32

Here's a random Ioniq PHEV, it can be both 4 and 8, so we can't remove the 8 signal def:

Can you guys try to replicate this so we can understand the meaning? Are there different sport modes? Try to toggle all the gears/sport options and post a route.

If we can verify that 8 is also sport, then we can merge 4 and 8

@sshane Brought back 8.

@sshane

Will there be anything else that is needed to merge this?

Here's a random Ioniq PHEV, it can be both 4 and 8, so we can't remove the 8 signal def:

Can you guys try to replicate this so we can understand the meaning? Are there different sport modes? Try to toggle all the gears/sport options and post a route.

If we can verify that 8 is also sport, then we can merge 4 and 8

Been a while but this came up again recently :D, I will need to test, but you asked

Are there different sport modes?

Technically, you can either have it on "S" or you can also have it on a specific gear selected (1 thru 6) and that's still sports mode

This PR has had no activity for 9 days. It will be automatically closed in 2 days if there is no activity.

This PR has been automatically closed due to inactivity. Feel free to re-open once activity resumes.